Skip to content

Use ASCII lower case for misc case folding #7511

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

tstarling
Copy link
Contributor

Use ASCII case conversion instead of locale-dependent case conversion in
the following places:

  • grapheme_stripos() and grapheme_strripos() in the "fast" path
  • ldap_get_entries()
  • oci_pconnect() for case folding of parameters when constructing a key into the connection or session pool
  • SoapClient: case folding of function names
  • get_meta_tags(): case conversion of property names
  • http stream wrapper: header names
  • phpinfo(): anchor names
  • php_verror(): docref URLs
  • rfc1867.c: Content-Type boundary parameter name
  • streams.c: stream protocol names

Bug fixes split out from https://wiki.php.net/rfc/strtolower-ascii

Use ASCII case conversion instead of locale-dependent case conversion in
the following places:

* grapheme_stripos() and grapheme_strripos() in the "fast" path
* ldap_get_entries()
* oci_pconnect() for case folding of parameters when constructing a key
  into the connection or session pool
* SoapClient: case folding of function names
* get_meta_tags(): case conversion of property names
* http stream wrapper: header names
* phpinfo(): anchor names
* php_verror(): docref URLs
* rfc1867.c: Content-Type boundary parameter name
* streams.c: stream protocol names
@tstarling
Copy link
Contributor Author

For grapheme_stripos() please refer to the discussion at #7506 (comment)

@nikic nikic closed this in c96be7b Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants